Skip to content
This repository has been archived by the owner on Sep 12, 2024. It is now read-only.

Update parser for payment requirement fields #26

Merged
merged 1 commit into from
Dec 22, 2021

Conversation

sgerrand
Copy link
Contributor

💁 Both payment_required_by and price_guarantee_expires_at are returned
from the API without microsecond precision:

"payment_requirements": {
  "requires_instant_payment": false,
  "price_guarantee_expires_at": "2021-12-24T11:39:45Z",
  "payment_required_by": "2021-12-25T11:39:45Z"
},

Both `payment_required_by` and `price_guarantee_expires_at` are returned
from the API without microsecond precision:

    "payment_requirements": {
      "requires_instant_payment": false,
      "price_guarantee_expires_at": "2021-12-24T11:39:45Z",
      "payment_required_by": "2021-12-25T11:39:45Z"
    },
@sgerrand sgerrand added the bug Something isn't working label Dec 22, 2021
@sgerrand sgerrand requested a review from jesse-c December 22, 2021 13:02
@sgerrand sgerrand enabled auto-merge December 22, 2021 13:07
Copy link
Contributor

@jesse-c jesse-c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Since these have been copied from our OpenAPI examples, we could add a ticket to correct them as well.

@sgerrand sgerrand merged commit 0c8ffdb into main Dec 22, 2021
@sgerrand sgerrand deleted the FLAPI-2064-fix-offer-payment-requirements-serialiser branch December 22, 2021 13:30
@sgerrand
Copy link
Contributor Author

I corrected the examples in https://github.com/duffelhq/platform/pull/7762

@sgerrand
Copy link
Contributor Author

Although we'll probably have to update everything else which flows on from that:

  • Postman collections (maybe)
  • API guides (maybe)
  • Something else?

@jesse-c
Copy link
Contributor

jesse-c commented Dec 22, 2021

Ah yes, that PR was connected to this. Yes to at least those other 2 places. I can't think of anywhere else.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants